From: Florian Müllner <fmuellner@gnome.org>
Date: Fri, 4 Oct 2024 11:10:35 +0000 (+0200)
Subject: tools/toolbox: Small refactor
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~173^2^2~227
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=1b8f67055b5b258ea282f9dc1f8d644524a7c747;p=gnome-shell.git

tools/toolbox: Small refactor

Handing the various options to different meson commands in meson-build
gets a bit tedious. Instead of handling them all on the toplevel,
split out *_command() functions that generate the commandline for
the corresponding meson command.

That way, options are handled locally where they matter, which
makes future changes and additions a bit easier.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3497>
---

diff --git a/tools/toolbox/meson-build.sh b/tools/toolbox/meson-build.sh
index fc17cb70e..cd9869d5c 100755
--- a/tools/toolbox/meson-build.sh
+++ b/tools/toolbox/meson-build.sh
@@ -37,9 +37,28 @@ find_toplevel() {
   done
 }
 
-needs_reconfigure() {
-  [[ ${#MESON_OPTIONS[*]} > 0 ]] &&
-  [[ -d $BUILD_DIR ]]
+setup_command() {
+  if [[ ${#MESON_OPTIONS[*]} > 0 && -d $BUILD_DIR ]]; then
+    RECONFIGURE=--reconfigure
+  fi
+
+  echo -n "meson setup --prefix=/usr $RECONFIGURE $WIPE ${MESON_OPTIONS[*]} $BUILD_DIR"
+}
+
+compile_command() {
+  echo -n "meson compile -C $BUILD_DIR"
+}
+
+install_command() {
+  echo -n "sudo meson install -C $BUILD_DIR"
+}
+
+dist_command() {
+  if [[ $RUN_DIST ]]; then
+    echo -n "meson dist -C $BUILD_DIR --include-subprojects"
+  else
+    echo -n :
+  fi
 }
 
 # load defaults
@@ -100,16 +119,12 @@ while true; do
   esac
 done
 
-find_toplevel
-
 BUILD_DIR=build-$TOOLBOX
-[[ $RUN_DIST ]] && DIST="meson dist -C $BUILD_DIR --include-subprojects" || DIST=:
 
-needs_reconfigure && RECONFIGURE=--reconfigure
+find_toplevel
 
 toolbox run --container $TOOLBOX sh -c "
-  meson setup --prefix=/usr $RECONFIGURE $WIPE ${MESON_OPTIONS[*]} $BUILD_DIR &&
-  meson compile -C $BUILD_DIR &&
-  sudo meson install -C $BUILD_DIR &&
-  $DIST
-"
+  $(setup_command) &&
+  $(compile_command) &&
+  $(install_command) &&
+  $(dist_command)"